Skip to content

picogame: enable on raspberry_pi_pico2 and raspberry_pi_pico2_w - #11357

Open
lynt-smitka wants to merge 1 commit into
adafruit:mainfrom
MakerClassCZ:pg-pico2
Open

lynt-smitka wants to merge 1 commit into
adafruit:mainfrom
MakerClassCZ:pg-pico2

Conversation

@lynt-smitka

Copy link
Copy Markdown

Enables picogame on raspberry_pi_pico2 and raspberry_pi_pico2_w.

The engine costs 44 KB and both fit at the port default -O3: pico2 keeps 89 KB free of its 1020 KB partition, pico2_w 16 KB of its 1536 KB.

The engine costs 44 KB here and both fit at the port default -O3: pico2
keeps 89 KB free of its 1020 KB partition, pico2_w 16 KB of its 1536 KB.

@dhalbert dhalbert left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could enable for most or all RP2350 boards, in mpconfigport.mk, as was done for RP2040 in #11358. But here it sounds like you don't need to change OPTIMIZATION_FLAGS.

@lynt-smitka

Copy link
Copy Markdown
Author

With the optimization in #11358 picogame can run on all RP2040 boards, because on each of these boards with default flags, saves over 150 kB of flash memory without any noticeable loss in performance, so size won’t be an issue. The RP2350 still has plenty of space.

In that case, I’d probably create an extra PR to enable picogame globally on these boards instead of this one and #11357.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants